Skip to content

Enable npm caching for faster linting workflows#5345

Merged
walterbender merged 1 commit intosugarlabs:masterfrom
varruunnn:workflowCache
Feb 1, 2026
Merged

Enable npm caching for faster linting workflows#5345
walterbender merged 1 commit intosugarlabs:masterfrom
varruunnn:workflowCache

Conversation

@varruunnn
Copy link
Contributor

@varruunnn varruunnn commented Jan 26, 2026

Summary
This PR optimizes the linter.yml workflow by enabling dependency caching.

Changes

  • Updated actions/setup-node@v4 to include cache: 'npm'.
  • also ran npx prettier for this file ( That's why its showing 42 deletions 43 additions the diff is only which i mentioned above)

Impact

  • Before: Dependencies were downloaded from the registry on every run (Slow).
  • After: Dependencies are restored from the GitHub cache (Fast).
  • Supporting Screenshot
Screenshot 2026-01-26 145232

Fixes
Closes #5344

@github-actions
Copy link
Contributor

✅ All Jest tests passed! This PR is ready to merge.

@varruunnn
Copy link
Contributor Author

@omsuneri Please review this pr

@varruunnn
Copy link
Contributor Author

@walterbender please review this pr

@walterbender walterbender merged commit d22a0f1 into sugarlabs:master Feb 1, 2026
6 checks passed
vanshika2720 pushed a commit to vanshika2720/musicblocks that referenced this pull request Feb 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Chore]: Enable npm dependency caching for ESLint workflow

2 participants